Merge remote-tracking branch 'origin/master' into webengine

Better handling of UTF-8 paths (#87)

* Better handling of UTF-8 paths

Remove gpsbabel calls to qPrintable which destroys non-ANSI path names on Windows. Replace with QString::toUtf8() and modify low-level file handling code to use _wfopen() and other wide-char functions on Windows to support non-ASCII paths. (I did an experiment to convince myself that fopen() doesn't do UTF-8, and that _wfopen() correctly creates a file with a non-ASCII filename starting from a UTF-8 name.) Add ufopen() function as a UTF-8 wrapper for fopen(). Change inifile_init() filename argument type to QString.

Leave serial port paths alone ("if your serial port path has non-ANSI characters, you're going to have a bad time").

* Fix valgrind mismatched free/malloc error.

* Correctly encode output paths in native locale on non-Windows

On Mac/Linux, convert output file path from internal UTF-8 to local encoding (which may be UTF-8 or something else) when creating the file. Fixes test_encoding failure.

* Support Unicode paths for .gz files on Windows

Use zlib's Windows-specific gzopen_w() to support Unicode paths.

* Convert UTF-8 to local encoding when opening gzip file on non-Windows

This should fix the test_encoding failure on Travis-CI.

Also, change Windows side to manually allocating wchar_t array; although a bit less elegant, it's very localized and avoids introducing a whole new dependency on std::wstring.

* Use explicit char* to QString conversion in qPrintable

The implicit conversion works, but I prefer the an explicit one (if only as a reminder that we should convert all internal strings to QString).

* Add comment to QString-to-wchar_t[] conversion

... since it may not be immediately obvious what that line of code does.

* Fix merge conflict resolution error

* Fix merge conflict (again)

* Revert shape.cc

* Update gbfile.cc

* Update mkt_logger.cc to use QString tempfile paths.

* Change ufopen() to take QString filename argument.
Where converting from char*, use an explicit QString::fromUtf8() conversion rather than the implicit QString(const char*) constructor.

* Use QFile::exists() and QFile::open() rather than ufopen() to verify a file exists and can be opened.

* Remove redundant call to QFile::exists(). Trivial include cleanup in main.cc.

* Remove superfluous QVector include.

* add newline at EOF.

Merge pull request #124 from tsteven4/appveyor

Appveyor for misc internal builds (debug /opt, 32/64, mscvc/gcc, etc) and start adding features to allow a full release withing Amveor, hopefully in lliest tientethan it takes my VM down download and register the mandatory 4GB of updates!  Hooray kl

Merge pull request #132 from viettaml/getenv-unicode

Support Unicode environment variables.

get gui map preview working with webengine.

Merge pull request #131 from gpsbabel/551

add missing Qt libs for gui on linux.

Support Unicode environment variables.
Create ugetenv() OS-abstraction to return a QString environment variable value. Use it in place of getenv() in most places. Update inifile.cc routines to use QString to support Unicode paths for the gpsbabel.ini file, also greatly streamlining the code.

add missing Qt libs for gui on linux.

The required plugin platforms/libqxcb.so needs
libQt5DBus and libQt5XcbQpa.

Merge pull request #5 from gpsbabel/master

Merge gpsbabel/gpsbabel to viettaml/gpsbabel

Merge pull request #130 from tsteven4/encoding_test

add a test of utf8 filename handling.

add the changes for the latin1 test.

enhance latin1 encoding test.

add a test of utf8 filename handlig.

Merge pull request #129 from viettaml/master

Fix KML writer crash bug in realtime trackign.

script windows release process.

address reviewer concerns wrt inno setup.

Fix KML writer crash bug
When Google Earth uses gpsbabel to import KML in "live" USB mode, kml_wr_position_init() is called before kml_wr_init() resulting in deferencing an uninitialized "writer" pointer. The solution is just to remove the setAutoFormatting(false) call; there's no real need for it as it doesn't measurably impact performance. Also remove other calls to remove nice formatting on timestamps (addressing a FIXME comment), and update the golden test data to match.

Merge pull request #125 from gpsbabel/cleanup

More strongly order header includes, some const-ification tweaks.

Merge branch 'master' into cleanup

Merge pull request #123 from tsteven4/shape

cleanup support for shape files.

Try hoisting windows.h earlier for windows build.

Remove more strays.

Eliminate stray .note files.

Try to make #includes consistently ordered.

update appveyor build.

run basic tests.
build windows installer.
archive installer.

delete obsolete translation file from gui.

mtk test workaround for windows.

Merge pull request #122 from gpsbabel/header_fixes

Remove remaining scraps of electric fence. Favor Valgrind to Efence.

Mechanically update c style foo(void)

cleanup support for shape files.
memory leaks, memory corruption, and other bugs fixed.
on write the required .dbf file is now created.
test added.
document updated.

Remove remaining scraps of electric fence. Valgrind > efence.

Merge pull request #121 from tsteven4/gpxtest

Input redirection can fail with gpx reader on windows

Merge pull request #114 from tsteven4/main_qstrings

Qstring-ification of main.cc.

update comment on QXmlStreamReader failures.

work around QXmlStreamReader issue on windows.

Merge pull request #120 from tsteven4/trackfilter_qsort

ensure track filter merge operation is consistent.

ensure track filter merge operation is consistent.

Merge pull request #119 from tsteven4/gtmtest

make gtm test OS independent.

make gtm test OS independent.

Merge pull request #118 from tsteven4/globalsat2

fix globalsat dump on windows.

fix globalsat dump on windows.

QString-ification of main reviewers suggestions.

Merge pull request #117 from tsteven4/linuxusb

fix GPSBabel.pro for linux adding -lusb.

fix GPSBabel.pro for linux adding -lusb.

Merge pull request #116 from tsteven4/msvc2

build using msvc x64 on AppVeyor.

build using msvc x64 on AppVeyor.

Qstring-ification of main.cc.

Merge pull request #113 from tsteven4/elevation_precision

change default gpx writer elevation precision to 3.

change default gpx writer elevation precision to 3.

Merge pull request #112 from tsteven4/gopal_date

use locale independent string for gopal tracklog.

use locale independent string for gopal tracklog.

More style/warning fixes.

Fix trackfilter.

Fix nmea.

Merge branch 'master' of https://github.com/gpsbabel/gpsbabel

Reapply warning fixes to skytraq.

Bring skytraq back to original version.

Merge pull request #111 from tsteven4/unicsv_digits

fix unicsv writer msec corruption issue.

fix unicsv writer msec corruption issue.

The final (?) piece for presubmit jooy..

Mmore of the code cldeanup handle. It was less fun than it looked.

More of the mechanical warning drugery between naps.

A bunch of busywork to satisfy hyperactive warnings from newer compilers. Part 1. Explictly initialize every member of a struct, Don't use C-style functions that take no args, don't name method arguments that aren't used, etc.

Merge pull request #108 from gpsbabel/header_fixes

Down-merge appveyor work in progress.

Merge pull request #110 from tsteven4/windows_testo

track-discard test to use testo gpsbabel()

track-discard test to use testo gpsbabel()

Merge pull request #109 from tsteven4/zlib

Update zlib from 1.2.8 to 1.2.11.

Update zlib from 1.2.8 to 1.2.11.

Keep the explict -l in LIBS.

Omit the explicit .lib

Try letting the compiler driver resolve path to setupapi.lib

Doink. GPSBabel.pro. app.pro is the GUI.

qmake with path instead of cd

Chdir into gpsbabel before build.

First take at appveyor. Inspired by VLC. This should run qmake.

Another deprecated file from MSVC proj. Qt Includes are still whacked.
Probably best to just recomment qmake as the Official Way.

First pass at removing obsolete files from MSVC proj.

Merge pull request #107 from tsteven4/test

enhance test_encoding to check gbfile gzapi.

enhance test_encoding to check gbfile gzapi.

Merge pull request #106 from tsteven4/test_encoding

enhance test_encoding to check up on gbfile.

enhance test_encoding to check up on gbfile.

Merge pull request #105 from tsteven4/nest_branches

fix bug with nested batch files.

fix bug with nested batch files.

Merge pull request #104 from tsteven4/lowranceusr4_overflow

fix potential buffer overflow in lowranceusr4 reader.

fix potential buffer overflow in lowranceusr4 reader.

Merge pull request #100 from gpsbabel/header_fixes

Remove case_ignore_str_match. Header cleanups.

Ref isnan from cstd namespace.

Merge pull request #99 from tsteven4/resync2

add test for encoding mangling of arguments, and fix the detected issues.

fix mangled argument handling on none utf-8 unix machines.

add encoding test for travis builds.

Remove leftover PDB code from the Palm/OS era.

Merge pull request #98 from gpsbabel/header_fixes

More header cleanups from trees laying around.

Typo fixes.

Fix hyperactive warnings in new clang/gcc versions.

Remove unused case_ignore_str_match.

More header cleanups from trees laying around.

More C++-like
Whitespace fixes
Include guards

Merge pull request #97 from gpsbabel/master

Minor Header fixes

Merge pull request #88 from viettaml/viettaml-patch-6

Add Unicode argument support to Windows

Merge pull request #95 from fbonzon/patch-001

Sync ICAO country code table with Wikipedia

Fix tests